keda-operator is also exposing metrics on default port 8080 like keda_scaler_, keda_scaled_objects_, keda_resource_* or keda_trigger_*. There are no annotations for that currently, we would need to add:

    prometheus.io/scrape: "true"
    prometheus.io/port: {{ .Values.prometheus.operator.port | quote }}
    prometheus.io/path: "/metrics"

in https://github.com/kedacore/charts/tree/main/keda/templates/manager/deployment.yaml
I also noticed that https://github.com/kedacore/charts/blob/main/keda/templates/manager/deployment.yaml#LL101C32-L101C32 is static, so if you change the value of prometheus.operator.port in the values.yaml, it may not work

nice catches!
Thanks for the feedback ❤️
I have to redo this PR soon, so I'll address them 😄

@JorTurFer could you please rebase this PR?

@JorTurFer could you please rebase this PR?

Yes, I'll do it during this week

I still got this pending but I'm out of the office :(

@JorTurFer could you please rebase this PR?

done!

PTAL @zroubalik , we need to merge this because the chart is "broken" until it due to the parameters change in MS

@JorTurFer @zroubalik @tomkerkhove my apologies for the multi-mention but I've just hit this and I was wondering if you're willing to push up a revision of the current KEDA chart to contain this and few other commits?

@JorTurFer @zroubalik @tomkerkhove my apologies for the multi-mention but I've just hit this and I was wondering if you're willing to push up a revision of the current KEDA chart to contain this and few other commits?

No worries for the mention 😄
We plan to cut a release next week: kedacore/keda#4795